Skip to content

Conversation

@sgiehl
Copy link
Member

@sgiehl sgiehl commented Dec 11, 2025

Description

  • Map client-side faults to consistent 4xx responses: missing/invalid API params return 400, invalid actions return
    404, missing chunks return 404, missing plugins return 404, deactivated plugins return 403.
  • Silence noisy error logging for 4xx-only failures by tagging these paths as HttpCodeException and downgrading log
  • System/integration tests added for HTTP status expectations (400 missing param, 404 missing plugin, 404 invalid
    action, 404 missing asset chunk) and for plugin activation checks (404 missing plugin, 403 deactivated) plus missing
    chunk error in the asset fetcher.

Checklist

  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules

Review

@sgiehl sgiehl added this to the 5.7.0 milestone Dec 11, 2025
@sgiehl sgiehl requested a review from a team December 11, 2025 11:17
@sgiehl sgiehl force-pushed the dev-19666 branch 3 times, most recently from e9a83d0 to 85a28c1 Compare December 11, 2025 16:54
@sgiehl sgiehl force-pushed the dev-19666 branch 3 times, most recently from ab2c9f9 to 951093c Compare December 19, 2025 09:29
@sgiehl sgiehl requested a review from mneudert December 19, 2025 10:13
@github-actions
Copy link
Contributor

github-actions bot commented Jan 3, 2026

If you don't want this PR to be closed automatically in 28 days then you need to assign the label 'Do not close'.

@github-actions github-actions bot added the Stale The label used by the Close Stale Issues action label Jan 3, 2026
mneudert
mneudert previously approved these changes Jan 8, 2026
@github-actions github-actions bot removed the Stale The label used by the Close Stale Issues action label Jan 9, 2026
@sgiehl sgiehl merged commit 04e57fc into 5.x-dev Jan 12, 2026
28 of 29 checks passed
@sgiehl sgiehl deleted the dev-19666 branch January 12, 2026 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants